Skip to content

Add Aliyun Elasticsearch CBOR transport and CLI - #835

Draft
vicety wants to merge 3 commits into
zilliztech:mainfrom
vicety:agent/aliyun-elasticsearch-query-encoding
Draft

vicety wants to merge 3 commits into
zilliztech:mainfrom
vicety:agent/aliyun-elasticsearch-query-encoding

Conversation

@vicety

@vicety vicety commented Aug 5, 2026 •

Copy link
Copy Markdown

What changed

  • Keep Base64 big-endian float32 JSON as the default query transport and add optional raw little-endian float32 CBOR queries.
  • Carry Basic authentication on raw CBOR transport requests, report HTTP errors, and close the worker client.
  • Add an aliyunelasticsearch CLI command, index-name configuration, environment overrides, and usage guidance for existing indexes.
  • Add the cbor2 optional dependency, documentation, and focused CBOR authentication/CLI regression tests.

Why

Aliyun Elasticsearch supports a raw-f32 CBOR query path, but low-level transport requests need explicit authentication. The CLI also makes it possible to benchmark an existing index without changing its mapping or shard topology.

Validation

  • The supplied patch reports 61 tests passed with Elasticsearch 8.16.0 / elastic-transport 8.17.1 and 61 tests passed with Elasticsearch 9.5.1 / elastic-transport 9.4.2; it also reports Ruff 0.12.12 and Black passed. These results were not independently rerun here.
  • Local Python syntax compilation and git diff --check passed.
  • No Elasticsearch performance rerun was performed.

@sre-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: vicety
To complete the pull request process, please assign xuanyang-cn after the PR has been reviewed.
You can assign the PR to them by writing /assign @xuanyang-cn in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vicety
vicety force-pushed the agent/aliyun-elasticsearch-query-encoding branch from c482285 to 4ab8ccb Compare August 11, 2026 23:27
@vicety vicety changed the title Support Aliyun Elasticsearch query encoding Add Aliyun Elasticsearch CBOR query transport Aug 11, 2026
@vicety vicety changed the title Add Aliyun Elasticsearch CBOR query transport Add Aliyun Elasticsearch CBOR transport and CLI Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants